Skip to content
This repository was archived by the owner on Oct 22, 2024. It is now read-only.

Conversation

@langleyd
Copy link
Member

@langleyd langleyd commented Sep 18, 2024

Resetting the captcha does in "componentWillUnmount" does not clear the challenge overlay from the body in android webviews(I confirmed it's getting called with a valid value for captchaWidgetId).

This pr adds a workaround to search for orphaned iframe and remove it from the body.
We can remove this when the "mobile_register" page is retired(i.e. MAS works with matrix.org).

@langleyd langleyd added the T-Task Tasks for the team like planning label Sep 18, 2024
@langleyd langleyd changed the title Try manually clear any left over recaptcha overalys to fix bug in android webview Try manually clear the left over recaptcha overaly to fix bug in android webview Sep 18, 2024
@langleyd langleyd changed the title Try manually clear the left over recaptcha overaly to fix bug in android webview Manually clear the left over recaptcha overaly to fix bug in android webview Sep 18, 2024
@langleyd langleyd changed the title Manually clear the left over recaptcha overaly to fix bug in android webview Manually clear orphaned recaptcha challenge overlay in android webviews Sep 18, 2024
@langleyd langleyd marked this pull request as ready for review September 18, 2024 14:41
@langleyd langleyd requested a review from a team as a code owner September 18, 2024 14:41
Copy link
Member

@dbkr dbkr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙈

@langleyd langleyd added this pull request to the merge queue Sep 18, 2024
Merged via the queue into develop with commit 154bf33 Sep 18, 2024
26 checks passed
@langleyd langleyd deleted the langleyd/fix_captcha_on_android branch September 18, 2024 17:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

T-Task Tasks for the team like planning

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants